-
Notifications
You must be signed in to change notification settings - Fork 250
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sizing Convenience Methods #62
base: master
Are you sure you want to change the base?
Conversation
They let you set vertical and horizontal values for all three in one line instead of always setting "top" and "bottom" or "left" and "right" values.
They let you set vertical and horizontal values for all three in one line instead of always setting "top" and "bottom" or "left" and "right" values.
very interesting, thanks for making this! the only concerns I have are:
and, of course:
|
personally I find this one a bit unnecessary, just because everyone is really used to |
I've actually never bothered to memorize the padding shorthands with 3 or 4 args... even the one with 2 args used to trip me up when I first started CSS. So I would make use of this... it's a bit more clear than |
i find most css stuff a bit counter-intuitive since people usually talk about dimensions x/y width/height, but css stuff is always clockwise from the top |
I think we don't really need it with padding-left etc |
Why this would be helpful: while you can use a shorthands like So, .foo
padding-top: 10px;
.bar
padding-x: 20px; aplied to So, I'm for renaming the props to |
Opening this again—sorry for the confusion/emails—because I had requested it from
master
instead of its feature branch and it was causing me headaches elsewhere.